-
Notifications
You must be signed in to change notification settings - Fork 1
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
⬆️ [#49] update dependencies #51
Conversation
.storybook/preview.ts
Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
https://github.com/open-formulieren/formio-builder/blob/main/.storybook/preview.ts also has some additional type annotations that can/should be used here
@@ -17,15 +17,15 @@ | |||
"scripts": { | |||
"build:esm": "tsc -p tsconfig.prod.json && tsc-alias -p tsconfig.prod.json", | |||
"build:cjs": "tsc -p tsconfig.prod.json --module commonjs --outDir dist/cjs && tsc-alias -p tsconfig.prod.json --outDir dist/cjs", | |||
"build:storybook": "build-storybook --webpack-stats-json", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
the --webpack-stats-json
was used for some reason 🤔
@@ -42,49 +42,49 @@ | |||
"homepage": "https://github.com/open-formulieren/formio-renderer#readme", | |||
"peerDependencies": { | |||
"formiojs": "~4.13.0 || ~4.14.0", | |||
"react": "^17.0.2" | |||
"react": "^18.2.0" | |||
}, | |||
"devDependencies": { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
feel free to make the dev tools setup the same as the formio builder. I'd rather see ESLint added to both repositories at the same time instead of this mixed set up now.
This repo was started from a very outdated boilerplate, while the formio-builder one was crafted by hand. You can be bold ;)
03ff5a8
to
a9ddcad
Compare
1d8e385
to
ada7ba5
Compare
6cab75e
to
4d5c640
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you!
No description provided.